Skip to content

refactor: CD 개선#217

Merged
clxxrlove merged 2 commits intodevelopfrom
refactor/#215
Mar 26, 2026
Merged

refactor: CD 개선#217
clxxrlove merged 2 commits intodevelopfrom
refactor/#215

Conversation

@clxxrlove
Copy link
Member

🔗 관련 이슈

📙 작업 내역

  • CI에서 시뮬레이터용으로 빌드된 캐시가 CD에서 사용되면서 CFBundleSupportedPlatforms [iPhoneSimulator] 에러 발생
  • skip-spm-cache 옵션 추가
  • CD 워크플로우에서 SPM/DerivedData 캐시 비활성화
  • 릴리즈 빌드는 항상 클린 빌드로 진행

💬 추가 설명 or 리뷰 포인트 (선택)

  • CI는 빨라야 하지만 CD는 안정성이 더 중요해서 캐시 다 날림 머지 후 결과 관찰해야할듯

@coderabbitai
Copy link

coderabbitai bot commented Mar 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 897ff94c-0bec-4a64-a17e-68981174f755

📥 Commits

Reviewing files that changed from the base of the PR and between ba78c03 and b025d4a.

📒 Files selected for processing (4)
  • .github/actions/setup-build-env/action.yml
  • .github/workflows/cd_develop.yml
  • .github/workflows/cd_main.yml
  • Projects/App/Project.swift

📝 Walkthrough

워크스루

GitHub Actions 워크플로우를 리팩토링하여 SPM 캐싱 로직을 조건부로 제어할 수 있는 새로운 입력값(skip-spm-cache)을 추가했습니다. 두 개의 배포 워크플로우에서 명시적 DerivedData 캐싱 단계를 제거하고 새로운 입력값으로 설정했습니다. 또한 애플리케이션 버전을 1.1.0에서 1.1.1로 업데이트했습니다.

변경 사항

코호트 / 파일 요약
GitHub Actions SPM 캐싱 리팩토링
.github/actions/setup-build-env/action.yml
새로운 입력값 skip-spm-cache를 추가하여 SPM 캐싱을 조건부로 제어합니다. SPM 캐시 단계와 의존성 설치 로직을 조건부로 실행하도록 변경하고, 캐시 상태 출력 메시지를 업데이트합니다.
배포 워크플로우 DerivedData 캐싱 제거
.github/workflows/cd_develop.yml, .github/workflows/cd_main.yml
명시적 DerivedData 캐싱 단계를 제거하고 setup-build-env 액션에 skip-spm-cache: "true" 옵션을 추가합니다.
애플리케이션 버전 업데이트
Projects/App/Project.swift
번들 단축 버전을 1.1.0에서 1.1.1로 업데이트합니다.

예상 코드 리뷰 노력

🎯 3 (보통) | ⏱️ ~20분

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/#215

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@clxxrlove clxxrlove merged commit a9e8db4 into develop Mar 26, 2026
5 checks passed
@clxxrlove
Copy link
Member Author

@coderabbitai summary

@github-actions github-actions bot requested a review from jihun32 March 26, 2026 03:50
@clxxrlove clxxrlove deleted the refactor/#215 branch March 26, 2026 03:50
@coderabbitai
Copy link

coderabbitai bot commented Mar 26, 2026

✅ Actions performed

Summary regeneration triggered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant